Skip to content

fix: main to dev downmerge#722

Closed
Ragini-Microsoft wants to merge 3 commits intodevfrom
main
Closed

fix: main to dev downmerge#722
Ragini-Microsoft wants to merge 3 commits intodevfrom
main

Conversation

@Ragini-Microsoft
Copy link
Contributor

Purpose

This pull request updates the infrastructure configuration for the content generation solution, focusing on restricting allowed deployment regions for Azure AI services and making a minor dependency ordering fix. The changes ensure that deployments are limited to supported regions and improve template consistency.

Region restriction and validation:

  • Added an explicit list of allowed Azure regions for AI service deployment in both main.bicep and main.json. This ensures that only supported regions (e.g., australiaeast, eastus2, uksouth, etc.) can be selected, aligning with the availability of specific AI models. [1] [2]

Infrastructure and template consistency:

  • Updated the ARM template hash in main.json to reflect the changes made in the Bicep template.

Dependency fix:

  • Corrected the order of dependencies for AI service resources in main.json to ensure proper deployment sequencing, specifically adjusting the placement of the openAI and cognitiveServices private DNS zones.

Does this introduce a breaking change?

  • Yes
  • No

Golden Path Validation

  • I have tested the primary workflows (the "golden path") to ensure they function correctly without errors.

Deployment Validation

  • I have validated the deployment process successfully and all services are running as expected with this change.

What to Check

Verify that the following are valid

  • ...

Other Information

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants